-
Notifications
You must be signed in to change notification settings - Fork 207
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
finalize 2.9.0 #1180
finalize 2.9.0 #1180
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@YamatoSecurity Thanks for your pull request.
I commented because I am a little concerned about some of the descriptions in CHANGELOG.
Could you please confirm this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!🚀
Co-authored-by: DustInDark <[email protected]>
Co-authored-by: DustInDark <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #1180 +/- ##
==========================================
- Coverage 83.82% 83.74% -0.08%
==========================================
Files 26 26
Lines 23388 23492 +104
==========================================
+ Hits 19604 19673 +69
- Misses 3784 3819 +35
☔ View full report in Codecov by Sentry. |
@hitenkoku Since it is a simple change and I want to release soon today I added the code for #1182 |
@YamatoSecurity thanks for your add. |
Finalized for 2.9.0.